Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tzutils: tzdata provides py3:tzdata #53607

Merged
merged 2 commits into from
Dec 23, 2024
Merged

Conversation

classabbyamp
Copy link
Member

according to the Python documentation, the zoneinfo module uses the system timezone data if available, and also provides an installable tzdata library on PyPi. Some Python packages (like pandas) spuriously always depend on this library. By having tzdata provide this py3: package, it would work around Python packages that make this mistake, and avoid needing to package python3-tzdata or set noverifypydeps (which would be overkill).

@sgn, this is your package, what do you think?

Testing the changes

  • I tested the changes in this PR: YES

@sgn
Copy link
Member

sgn commented Dec 21, 2024

It doesn't matter anyway, since our python3-tzdata is almost empty and uses system zoneinfo.

@classabbyamp
Copy link
Member Author

oh, python3-pytzdata?

@classabbyamp
Copy link
Member Author

that's not the same thing as tzdata i think

@sgn
Copy link
Member

sgn commented Dec 21, 2024

Ah yes! Anyway, what is different between: provides="python3-tzdata-${version}_1" and provides="py3:tzdata-${version}_1"?

@sgn
Copy link
Member

sgn commented Dec 21, 2024

/me removes pytzdata too! #53609

@classabbyamp
Copy link
Member Author

what is different between: provides="python3-tzdata-${version}_1" and provides="py3:tzdata-${version}_1"?

py3: will get used by the python dependency verification hook, but we could have it provide both

classabbyamp and others added 2 commits December 23, 2024 10:36
according to the Python documentation[1], the `zoneinfo` module uses the
system timezone data if available, and also provides an installable `tzdata`
library on PyPi. Some Python packages (like pandas) spuriously always
depend on this library. By having `tzdata` provide this py3: package,
it would work around Python packages that make this mistake, and avoid
needing to package `python3-tzdata` or set `noverifypydeps` (which would
be overkill).

[1]: https://docs.python.org/3/library/zoneinfo.html

Close: void-linux#53607
We don't care if 2 timezones provides same zoneinfo as of current time
being.
@sgn sgn merged commit 99a3973 into void-linux:master Dec 23, 2024
8 checks passed
sgn pushed a commit that referenced this pull request Dec 23, 2024
according to the Python documentation[1], the `zoneinfo` module uses the
system timezone data if available, and also provides an installable `tzdata`
library on PyPi. Some Python packages (like pandas) spuriously always
depend on this library. By having `tzdata` provide this py3: package,
it would work around Python packages that make this mistake, and avoid
needing to package `python3-tzdata` or set `noverifypydeps` (which would
be overkill).

[1]: https://docs.python.org/3/library/zoneinfo.html

Close: #53607
@classabbyamp classabbyamp deleted the py3-tzdata branch December 23, 2024 03:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants